Вернул прежнее величие главе персонала №3 #184
Conversation
|
Warning Rate limit exceeded@DuckierReaper has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 25 minutes and 50 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
Обзор измененийWalkthroughИзменения касаются конфигурации роли "Начальника персонала" в файле YAML. Ключевые модификации включают уменьшение веса должности с 20 до 10, обновление списка доступа к различным отделам и помещениям, а также корректировку стартового снаряжения. Добавлены новые разрешения для доступа в Chemistry, Engineering, Research, Security и другие отделы, а также для новых ролей. Изменения затрагивают как параметры доступа, так и начальное оборудование роли. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
3 попытка |
|
это не полный список всех доступов для расширенного. Глянь через консоль ID карт и добавь остальные |
Добавил главе персонала почти все доступы, за исключением оружейной, глав ( исключение КМ) и судебных ролей
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Resources/Prototypes/Roles/Jobs/Command/head_of_personnel.yml (2)
Line range hint
82-85: Улучшите стиль комментариев в кодеИзменения экипировки логично соответствуют сервисной роли, однако комментарий "plebe shoes" выглядит непрофессионально.
Предлагаю заменить комментарий на более профессиональный:
- shoes: ClothingsShoesLeather # DeltaV - HoP needs something better than plebe shoes. + shoes: ClothingsShoesLeather # DeltaV - Updated shoes to match role requirements
Line range hint
1-85: Обновите документацию ролиЗначительные изменения в конфигурации роли Начальника Персонала требуют обновления соответствующей документации:
- Новые права доступа
- Изменение концепции роли на сервисную
- Обновленное начальное снаряжение
Желаете, чтобы я помог составить обновленную документацию для роли?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/Roles/Jobs/Command/head_of_personnel.yml(1 hunks)
🔇 Additional comments (1)
Resources/Prototypes/Roles/Jobs/Command/head_of_personnel.yml (1)
37-67: Проверьте обоснованность доступа к службе безопасности
Расширение прав доступа соответствует результатам голосования, однако доступ к отделу безопасности (Security, Brig) может противоречить концепции сервисной роли. Рекомендуется дополнительно подтвердить необходимость этих разрешений.
✅ Verification successful
Доступ к службе безопасности обоснован и соответствует текущей конфигурации
Анализ конфигурационных файлов показывает, что:
- Доступ HoP к Security и Brig является частью общей реструктуризации роли
- В комментариях явно указано, что это изменение связано с переопределением HoP как сервисной роли ("HoP is a service role")
- Другие роли, такие как юрист, также имеют доступ к каналам безопасности и соответствующие разрешения
- Доступ к отделу безопасности намеренно ограничен базовым уровнем (нет доступа уровня HeadOfSecurity)
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Search for any existing security-related restrictions or policies
rg -i "hop.*security" --type yaml
rg -i "headofpersonnel.*security" --type yaml
Length of output: 83
Script:
#!/bin/bash
# Search for security-related access configurations in YAML files
rg "Security|Brig" --type yaml -C 5 Resources/Prototypes/Roles/Jobs/
# Search for any HoP role definitions and their access configurations
fd "head_of_personnel" --type f --extension yml --exec cat {}
Length of output: 49457
|
Добавил недостающие кроме глав (исключение КМ), оружейной, судебных ролей |
С доступами сделал. Еще вопрос касательно частот его гарнитуры, их тоже возвращать как прежде? Поставить картридж как у капитана |
| # Delta V - fuck all of this HoP is a service role | ||
| # - Chemistry | ||
| # - Engineering | ||
| # - Research | ||
| # - Detective | ||
| # - Salvage | ||
| # - Security # NoooOoOo!! My HoPcurity!1 | ||
| # - Brig | ||
| - Chemistry | ||
| - Engineering | ||
| - Research | ||
| - Detective | ||
| - Salvage | ||
| - Security # NoooOoOo!! My HoPcurity!1 | ||
| - Brig | ||
| - Lawyer | ||
| - Cargo | ||
| # - Atmospherics | ||
| # - Medical | ||
| - Atmospherics | ||
| - Medical |
There was a problem hiding this comment.
Тут стоит убрать дельтавишный комент и пометить марками # WD EDIT
|
Понял, сейчас залью |
Co-authored-by: Spatison <[email protected]>
Co-authored-by: Spatison <[email protected]>
Co-authored-by: Spatison <[email protected]>
|
Вроде сделал |
Убрал решетки там где не надо, вернув недостающие, по итогам голосования — 26.11.2024 5:15, доступы ( расширенные )
Описание PR
Добавляет ХоПу доступы, почти, во все углы станции. По итогам голосования — 26.11.2024 5:15
Изменения
🆑 ДикПикер